Skip to content

fix(ci): give the release-notes job a repo to talk to#283

Merged
OpenSauce merged 1 commit into
mainfrom
fix/notes-job-repo
Jul 26, 2026
Merged

fix(ci): give the release-notes job a repo to talk to#283
OpenSauce merged 1 commit into
mainfrom
fix/notes-job-repo

Conversation

@OpenSauce

Copy link
Copy Markdown
Owner

Follow-up to #282. The Windows upload fix worked — all three plugin bundles are attached to v0.3.0 — but the new notes job failed:

gh release view "$RELEASE_TAG" --json body --jq '.body'
failed to run git: fatal: not a git repository (or any of the parent directories): .git

The job deliberately has no actions/checkout (it only edits a release, it needs no source), so gh had no git remote to infer the repository from. GH_REPO tells it directly, which is cheaper than checking out the tree just to name a repo.

Only the notes step was affected — every bundle upload in that run succeeded, including the Windows one repaired by #282's shell: bash fix.

v0.3.0 status

Complete and published. All four archives attached, and the plugin download table was applied by hand with the same content and marker the job generates, so a future re-dispatch for that tag will correctly skip it as already present.

This change makes the next release do it unattended.

Verification

Same limitation as #282: the notes job only runs off a real release, so this gets its first live exercise at v0.4.0. The generated markdown itself is verified — it's exactly what was published to v0.3.0.

The notes job runs without actions/checkout, so gh had no git remote to
infer the repository from and died with "fatal: not a git repository"
before it could edit the release. GH_REPO tells gh directly, which is
cheaper than checking out the tree just to name a repo.

The bundle uploads in the same run all succeeded, including the Windows
one that the shell: bash fix repaired, so only the notes step was
affected. v0.3.0's notes were applied by hand; this makes the next
release do it on its own.
Copilot AI review requested due to automatic review settings July 26, 2026 12:55

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Fixes the plugin-release.yml release-notes (notes) job by explicitly telling the GitHub CLI which repository to target, avoiding the need for a checkout just to infer a git remote.

Changes:

  • Set GH_REPO: ${{ github.repository }} for the “Append plugin download table” step so gh release view/edit works without a .git directory.
  • Add inline comments documenting why GH_REPO is required in this job.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@OpenSauce
OpenSauce enabled auto-merge (squash) July 26, 2026 12:56
@OpenSauce
OpenSauce merged commit 3f81b08 into main Jul 26, 2026
11 checks passed
@OpenSauce
OpenSauce deleted the fix/notes-job-repo branch July 26, 2026 12:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants